Skip to content

Fix breakage v3.0 caused to the InterSystems Testing Manager extension #1619

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

gjsjohnmurray
Copy link
Contributor

This PR fixes #1618

@isc-bsaviano
Copy link
Contributor

isc-bsaviano commented Jul 18, 2025

@gjsjohnmurray Can you explain why this is necessary? Why can't the testing manager sync the files itself?

@gjsjohnmurray
Copy link
Contributor Author

I'd prefer not to have to make the Testing Manager have to deal with /api/atelier file- and directory-type endpoints directly. But perhaps I can make it talk to isfs://server/_vscode/NAMESPACE/foo.bar?csp URLs directly instead of using isfs://server/.vscode/foo.bar and relying on the the redirection (which is what's no longer happening for client-type workspaces since #1479). I'll try this out in Testing Manager.

@gjsjohnmurray gjsjohnmurray changed the title Fix breakage v3.0 caused the InterSystems Testing Manager extension Fix breakage v3.0 caused to the InterSystems Testing Manager extension Jul 18, 2025
@isc-bsaviano
Copy link
Contributor

Yeah, I can see why it would be easier to use vscode.workspace.fs on isfs URIs. I think using /_vscode directly should work though, assuming there really is a /_vscode web app on that server.

@gjsjohnmurray
Copy link
Contributor Author

I worked around it in the other extension so I'm closing this unmerged.

@gjsjohnmurray gjsjohnmurray deleted the fix-1618 branch July 18, 2025 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

v3.0 broke the InterSystems Testing Manager extension's operation when used with client-side editing paradigm
2 participants